lib: Always checksum content in deltas
authorColin Walters <walters@verbum.org>
Mon, 5 Dec 2016 22:22:46 +0000 (17:22 -0500)
committerAtomic Bot <atomic-devel@projectatomic.io>
Tue, 6 Dec 2016 15:59:35 +0000 (15:59 +0000)
commit4c8fc92aa02dbbd474009b61ca54cf49ef29da56
tree527918c552180f036c90b6228c51b7c59912c965
parent66da1199f0e6bff78b94d54b619c1bc2eed009c6
lib: Always checksum content in deltas

This is a follow up to conversation on list - in practice, if we're
backing away from summary signing, then it makes sense to remove the
special casing for checksums in deltas around summary signatures.

This is also related to the recent change to enable GPG checking for
commits in deltas - now we have a more coherent story between the
previous pull path and deltas.

I didn't do any performance checking, and while it's slightly annoying
that we're now doing sha256 on the delta content twice (once for the
part and once per object)...sha256 is pretty fast, I think most users
are I/O bound anyways, and it'd drop even farther if we started using
openssl.

Closes: #612
Approved by: jlebon
src/libostree/ostree-repo-pull.c
src/libostree/ostree-repo-static-delta-core.c
src/libostree/ostree-repo-static-delta-private.h
src/libostree/ostree-repo-static-delta-processing.c